[patch] md/raid10: double lock typo in handle_read_error()

This should be a spin_unlock_irq() instead of spin_lock_irq()

Signed-off-by: Dan Carpenter <error27 [at] gmail.com>

diff --git a/drivers/md/raid10.c b/drivers/md/raid10.c
index 9214e08..8b29cd4 100644
--- a/drivers/md/raid10.c
+++ b/drivers/md/raid10.c
[at] [at] -2117,7 +2117,7 [at] [at] read_more:
mbio->bi_phys_segments = 2;
else
mbio->bi_phys_segments++;
- spin_lock_irq(&conf->device_lock);
+ spin_unlock_irq(&conf->device_lock);
generic_make_request(bio);
bio = NULL;

--
To unsubscribe from this list: send the line "unsubscribe linux-raid" in
the body of a message to majordomo [at] vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Dan Carpenter [ Mi, 27 Juli 2011 11:52 ] [ ID #2062671 ]

Re: [patch] md/raid10: double lock typo in handle_read_error()

On Wed, 27 Jul 2011 12:52:02 +0300 Dan Carpenter <error27 [at] gmail.com> wrote:

> This should be a spin_unlock_irq() instead of spin_lock_irq()
>
> Signed-off-by: Dan Carpenter <error27 [at] gmail.com>
>
> diff --git a/drivers/md/raid10.c b/drivers/md/raid10.c
> index 9214e08..8b29cd4 100644
> --- a/drivers/md/raid10.c
> +++ b/drivers/md/raid10.c
> [at] [at] -2117,7 +2117,7 [at] [at] read_more:
> mbio->bi_phys_segments = 2;
> else
> mbio->bi_phys_segments++;
> - spin_lock_irq(&conf->device_lock);
> + spin_unlock_irq(&conf->device_lock);
> generic_make_request(bio);
> bio = NULL;
>

Thanks!
I haven't yet submitted the patch which introduced that bug, so I just fixed
the patch directly and credited you.

Thanks,
NeilBrown
--
To unsubscribe from this list: send the line "unsubscribe linux-raid" in
the body of a message to majordomo [at] vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
NeilBrown [ Mi, 27 Juli 2011 12:59 ] [ ID #2062675 ]
Linux » gmane.linux.raid » [patch] md/raid10: double lock typo in handle_read_error()

Vorheriges Thema: standard performance (write speed 20Mb/s)
Nächstes Thema: Repairing a Raid-6 array